azurerm_network_watcher_flow_log
- add nil check for traffic_analytics
#28416
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Community Note
Description
When
traffic_analytics
is removed, expandNetworkWatcherFlowLogTrafficAnalytics would always get the first item. So it causes crash. So I added nil check in this expand func.PR Checklist
For example: “
resource_name_here
- description of change e.g. adding propertynew_property_name_here
”Changes to existing Resource / Data Source
Testing
Note: Below failed test case is also failed with same error on Teamcity Daily Run. So it's not related with this PR.
![image](https://private-user-images.githubusercontent.com/19754191/400379868-5fcdb3d7-f224-419c-b75e-2ec4aacc4777.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkzMjkzODEsIm5iZiI6MTczOTMyOTA4MSwicGF0aCI6Ii8xOTc1NDE5MS80MDAzNzk4NjgtNWZjZGIzZDctZjIyNC00MTljLWI3NWUtMmVjNGFhY2M0Nzc3LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMTIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjEyVDAyNTgwMVomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTg5YmI0MmM3MWNlNDI4ZTIyZjQ0YzAyYzUwMmE1MmE3MGQwNGRiYzVlOTE0OWUyYmRlZGIxYjU4ZWQzM2Q0MWImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.O_hchaAQ4lvnyjvYCVXpJITeCwntLqf8Xv4iB6N3m-c)
Change Log
Below please provide what should go into the changelog (if anything) conforming to the Changelog Format documented here.
azurerm_network_watcher_flow_log
- add nil check fortraffic_analytics
This is a (please select all that apply):
Related Issue(s)
Fixes #28407
Note
If this PR changes meaningfully during the course of review please update the title and description as required.